Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(api): Expose defaults #358

Merged
merged 1 commit into from
Jan 11, 2018
Merged

feat(api): Expose defaults #358

merged 1 commit into from
Jan 11, 2018

Conversation

obartra
Copy link
Collaborator

@obartra obartra commented Jan 9, 2018

  • Export defaults
  • Replace "browser" with "unpkg" entry file in package.json to correctly
    import dev version in react while preserving unpkg compatibility.
  • Update dependencies (except storybook)

Closes #357 #355 #342

@obartra obartra force-pushed the expose-defaults branch 2 times, most recently from d7cba67 to e2595ae Compare January 9, 2018 05:28
Copy link
Collaborator

@arahansen arahansen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

circle.yml Outdated
@@ -22,6 +22,7 @@ test:
override:
- yarn build
- scripts/citests.sh:
timeout: 1200
parallel: true
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What issue were you seeing here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm still debugging this, this is not the right solution though.

iphone7 tests were timing out (still are) I thought we had too many stories but looking at saucelabs it seems like it's not completing correctly and re-trying.

It may be related to the picturebook update

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like the issue is related to the storybook update and iphone 7. I'll look into it separately #359

@obartra obartra force-pushed the expose-defaults branch 2 times, most recently from 2963679 to f989f2b Compare January 11, 2018 06:08
- Export defaults
- Replace "browser" with "unpkg" entry file in package.json to correctly
  import dev version in react while preserving unpkg compatibility.
- Update dependencies

Closes #357 #355
@obartra obartra merged commit 065d4aa into master Jan 11, 2018
@obartra obartra deleted the expose-defaults branch January 11, 2018 06:37
@obartra
Copy link
Collaborator Author

obartra commented Sep 6, 2018

🎉 This PR is included in version 16.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants